-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Composer improvements #3751
Composer improvements #3751
Conversation
soullivaneuh
commented
May 1, 2016
- Use new composer constraints notation
- Add missing PHP constraint
- New sonata project majors requiring
* Use new composer constraints notation * Add missing PHP constraint * New sonata project majors requiring
@sonata-project/contributors Can anyone explains me this issue? https://travis-ci.org/sonata-project/SonataAdminBundle/jobs/127012710#L554 |
1243ca7
to
b90cfa9
Compare
"symfony/yaml": "~2.3|~3.0", | ||
"jms/di-extra-bundle": "^1.7", | ||
"sensio/generator-bundle": "^2.3 || ^3.0", | ||
"symfony/yaml": "^2.3 || ^3.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting the sort-package
option in all composer.json of all sonata-projects would be great.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What sort-package
option does exactly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It sorts all packages alphabetically, but this works with composer require
only. https://getcomposer.org/doc/06-config.md#sort-packages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it works with composer validate
too?
Can be discussed on a sonata-project/dev-kit issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it works with composer validate too?
I tested that, but no, validate
does not report anything about that.